PR #1124 Review — feat(memory): add memory_summary tool for memory overview and source tracking

本 PR 添加 memory_summary 工具，扫描整个记忆存储并返回结构化概览：总数、自动/手动创建统计、分类分布、高频概念、高频文件引用和最近记忆列表。253 行新增，5 文件变动。

代码整体可读，frontmatter 解析和排序逻辑清晰，MemorySummary 结构设计合理。

发现

1. src/agent-memory/src/tools/memory_summary_tool.rs:~178 — Source tracking 依赖 4 个 source 值，但当前代码库中没有任何代码写入这些值 (CONFIRMED, 中)

match source.as_str() {
"auto-consolidation" | "auto-capture" => summary.auto_created += 1,
"manual-observe" | "manual-write" => summary.manual_created += 1,
_ => summary.unknown_source += 1,
}

在当前 main 分支中 grep 确认：auto-consolidation、auto-capture、manual-observe、manual-write 这四个字符串均不被任何代码写入 frontmatter。memory_observe 不写 source 字段（PR #1123 才添加但未合并），consolidation 代码也不写。结果是：合并后 所有
记忆都会被归为 unknown_source，auto_created 和 manual_created 恒为 0。Summary 的核心卖点——区分自动/手动来源——完全无法工作，直到多个后续 PR 合并。

2. src/agent-memory/src/tools/memory_summary_tool.rs:~142 — 扫描不排除 MEMORY.md 等非记忆文件 (CONFIRMED, 中)

PR #1122 在 mount root 创建 MEMORY.md 索引文件。本 PR 的 WalkDir 扫描包含所有 .md 文件，仅排除 meta_dir。MEMORY.md 没有 frontmatter，会被计入 total_memories（+1）、unknown_source（+1）、by_category["uncategorized"]（+1），甚至可能出现在
recent_memories 列表中。同理，若 mount root 存在 README.md 或其他非记忆 .md 文件也会被错误统计。应显式跳过已知的非记忆文件（至少跳过 MEMORY.md）。

3. src/agent-memory/src/tools/memory_summary_tool.rs:~89 — parse_frontmatter_flat 第 N 份拷贝 (CONFIRMED, 低-中)

这个函数在 PR #1120（session_history.rs）、PR #1122（memory_index.rs）、PR #1124（本 PR）中各有一份几乎完全相同的实现。如果三个 PR 都合并，代码库中将存在 3 个独立的 frontmatter 解析器。应提取为共享模块（如
crate::frontmatter::parse_flat），避免未来修复需同步多处。

4. src/agent-memory/tests/profile_test.rs:~322 — 注释与常量值不一致 (CONFIRMED, 低)

// 11 Tier A + 3 Tier B + 3 snapshot + 2 git + 2 consolidation + 4 task + 2 export/import = 27
const TOTAL_TOOLS: usize = 28;

注释仍然写 "= 27" 但值已改为 28。应更新为 + 1 summary = 28。

5. src/agent-memory/src/tools/memory_summary_tool.rs:~109 — parse_frontmatter_list 不支持 YAML 多行列表格式 (PLAUSIBLE, 低)

解析器假设列表值为内联 JSON 风格（["a", "b"]）。若 frontmatter 使用 YAML 标准多行格式：
concepts:
- rust
- memory
flat parser 的 !key.starts_with('-') 过滤会丢弃列表项，concepts 的值变为空字符串。top_concepts 统计会遗漏这些记忆的概念。实际风险取决于 consolidation 代码输出的格式——若统一使用内联风格则无影响。

LGTM